-
Notifications
You must be signed in to change notification settings - Fork 4k
ARROW-10350: [Rust] Fixes to publication metadata in Cargo.toml #9031
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
2406448 to
b519716
Compare
b519716 to
e22819a
Compare
|
@huitseeker Unfortunately the release verification scripts will need updating to reflect the changes to these Cargo.toml files. If you look at the CI failure you should be able to reproduce this locally by running the same script. |
|
I'll complete this during the week |
a6762a2 to
100fbd9
Compare
|
@nevi-me @andygrove The release verification component should now be fixed. Besides that, some integration tests seem to be running on the wrong side of a source incompatible change (e.g. the same compilation errors are met in the unrelated #9137). Those failures should probably be addressed in a distinct PR? |
|
Hey @huitseeker, the errors in integration tests should be fixed now, may you please kindly rebase |
100fbd9 to
d578bbd
Compare
paddyhoran
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Cleans some TOML fields and marks some testing crates not-for-publication.